Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tell dconf-edit about relocatable schemas #1355

Merged
merged 4 commits into from
May 17, 2023

Conversation

dnk
Copy link
Contributor

@dnk dnk commented Feb 18, 2023

support for dconf-editor relocatable schemas

@dnk
Copy link
Contributor Author

dnk commented Feb 18, 2023

here result of applying relocatable schema
image

@dnk dnk force-pushed the dconf-relocatable-schemas branch 3 times, most recently from ef7481b to 6dc1bb4 Compare February 18, 2023 20:54
@zhuyaliang
Copy link
Member

@dnk I don't quite understand the meaning of the commit message. Can you supplement the information and compress the submission

@dnk
Copy link
Contributor Author

dnk commented Feb 19, 2023

@zhuyaliang these changes are evolution of mate-desktop/mate-desktop#300
dconf-editor did not use gsettings and cannot get information about relocatable schemas. which cause inability to view/edit mate-applets' preferences (path /org/mate/panel/objects//prefs) in dconf-editor.

dconf-editor have ca.desrt.dconf-editor.Settings:relocatable-schemas-user-paths to specify users' relocatable schemas

gsettings describe ca.desrt.dconf-editor.Settings relocatable-schemas-user-paths

A dictionary that maps schema IDs with path specifications. It is used to allow the user to >associate a relocatable schema to certain paths. Path specifications may contain wildcards in >the form of empty segments (e.g. /ca/desrt/dconf-editor//), defining possibly multiple paths. >The same schema ID may be associated with multiple path specifications.`

this pull request update ca.desrt.dconf-editor.Settings:relocatable-schemas-user-paths to allow dconf-editor to use information about users' relocatable schema and to show it properly

@zhuyaliang
Copy link
Member

@dnk Thank you for your explanation. I can test it now

dnk added 4 commits February 21, 2023 18:44
remove duplicated schemas for currect path
remove incorrect schemas for current path
add "transfer full" annotation to make this method available throught
introspection
Copy link
Member

@lukefromdc lukefromdc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Builds and runs without any obvious problems, but I'm not the best reviewer for this one.

This cannot be tested for function with older versions of dconf-editor as they do not support relocatable schemas. I normally keep dconf-editor locked to version 3.18 as I have difficulty with the UI on newer versions. Testing with 3.36 this appeared to work, the name of the schema came up, and as a simple test I was able to lock the clock applet to the panel from it but wasn't able to find the custom font prefs from #1358
which I had build with this.

Assuming that's a problem with the other PR or just an inability of myself to navigate the newer dconf-editor UI to find it's changes so approving this PR as its functionality seems to work in newer versions of dconf-editor.

At some point I might need to try reverting all the UI changes from dconf-editor but keeping the backend updates in a local repo, but that might be unresolvable without applying commits by hand after all these years. Might be easier to just backport relocatable schema support to the older dconf, though so far I have been able to get by without it

@L-U-T-i
Copy link

L-U-T-i commented Mar 24, 2023

@lukefromdc

wasn't able to find the custom font prefs from #1358

I've rebuilt mate-panel with both PRs (#1355 and #1358), and I can find both settings "custom-font" and "use-custom-font" in "/ org / mate / panel /objects / clock / prefs" (dconf-editor-3.38.3-3.el9.x86_64).

@lukefromdc
Copy link
Member

lukefromdc commented Mar 24, 2023 via email

@dnk
Copy link
Contributor Author

dnk commented Mar 24, 2023

these changes went into libmate-panel-applet-4.so
to see result it is need to relaunch applets with updated libmate-panel-applet-4.so

@lukefromdc
Copy link
Member

lukefromdc commented Mar 25, 2023 via email

Copy link
Member

@raveit65 raveit65 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM,
the relocatable schema name is now visible in dconf-editor.
I recall in the beginning of MATE when gsettings port was done by Stefano that there was a problem with relocatable schemas.
Good to remove the hack to use such schemas.

@raveit65
Copy link
Member

raveit65 commented May 1, 2023

This will also detect old non usable forgotten settings in relocatable-schemas which can be removed.
So +1 for merging it.

@raveit65
Copy link
Member

Is there another reviewer or should i merge it?

@lukefromdc lukefromdc merged commit 277418c into mate-desktop:master May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants